RATY-357 stuck Dependabot workflows#481
Conversation
|
Warning Review limit reached
Next review available in: 49 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR migrates ESLint configuration from eslint-plugin-react/eslint-plugin-import to ChangesLint tooling migration and source annotations
Estimated code review effort: 2 (Simple) | ~15 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
vite.config.ts (1)
4-5: 🚀 Performance & Scalability | 🔵 Trivial | ⚡ Quick winPrefer
@vitejs/plugin-react-swcunless Babel-only transforms are required
@vitejs/plugin-react-swcis Vite 8-compatible and keeps JSX transforms on the faster SWC path. Theresolve.tsconfigPathsand ESLint 10 concerns don’t block this setup, so the main tradeoff here is the React plugin swap.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@vite.config.ts` around lines 4 - 5, The Vite React setup is using the Babel-based React plugin, but this should be switched to the SWC-based plugin for better performance and Vite 8 compatibility. Update the import and plugin usage in vite.config.ts from the current React plugin to `@vitejs/plugin-react-swc`, and keep the rest of the Vite config unchanged unless a Babel-only transform is actually needed.eslint.config.mjs (1)
33-71: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick winRe-enable
@eslint-react/dom-no-dangerously-set-innerhtmlin the cleanup pass. It’s the only disabled rule here that directly guards against XSS;recommended-latestanddisable-conflict-eslint-plugin-react-hooksare valid presets in the locked package versions.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@eslint.config.mjs` around lines 33 - 71, The cleanup override in eslint.config.mjs is disabling `@eslint-react/dom-no-dangerously-set-innerhtml` along with unrelated noisy rules, but this rule should be re-enabled. Update the rules block in the config section that applies to the `@eslint-react` and react-hooks presets so dom-no-dangerously-set-innerhtml is not turned off, while keeping the valid recommended-latest and disable-conflict-eslint-plugin-react-hooks preset usage intact.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Around line 37-58: The Vitest coverage dependency is out of sync with the
installed Vitest version, so update the `@vitest/coverage-v8` entry in
`package.json` to match the same patch line as `vitest`, or bump `vitest` and
the coverage plugin together so both stay aligned. Use the package names
`vitest` and `@vitest/coverage-v8` to locate and keep the test toolchain
versions consistent.
---
Nitpick comments:
In `@eslint.config.mjs`:
- Around line 33-71: The cleanup override in eslint.config.mjs is disabling
`@eslint-react/dom-no-dangerously-set-innerhtml` along with unrelated noisy rules,
but this rule should be re-enabled. Update the rules block in the config section
that applies to the `@eslint-react` and react-hooks presets so
dom-no-dangerously-set-innerhtml is not turned off, while keeping the valid
recommended-latest and disable-conflict-eslint-plugin-react-hooks preset usage
intact.
In `@vite.config.ts`:
- Around line 4-5: The Vite React setup is using the Babel-based React plugin,
but this should be switched to the SWC-based plugin for better performance and
Vite 8 compatibility. Update the import and plugin usage in vite.config.ts from
the current React plugin to `@vitejs/plugin-react-swc`, and keep the rest of the
Vite config unchanged unless a Babel-only transform is actually needed.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 42622371-4476-4e4e-a5d7-d94e197ee0da
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (13)
eslint.config.mjspackage.jsonpnpm-workspace.yamlsrc/common/components/modal/BaseDialogModal.tsxsrc/common/components/notificationsContext/NotificationsContext.tsxsrc/common/components/table/Table.tsxsrc/common/components/textInput/InputWrapper.tsxsrc/domain/app/routes/SilentRenewRedirect.tsxsrc/domain/event/eventForm/useEventFormSubmitRequests.tsxsrc/domain/venue/venueDataFields/VenueDataFields.tsxsrc/playwright/testWithFixtures.tstsconfig.jsonvite.config.ts
💤 Files with no reviewable changes (3)
- src/domain/app/routes/SilentRenewRedirect.tsx
- src/domain/venue/venueDataFields/VenueDataFields.tsx
- src/playwright/testWithFixtures.ts
|
|
KULTUS-ADMIN-UI branch is deployed to platta: https://kultus-admin-ui-pr481.dev.hel.ninja 🚀🚀🚀 |
1 participant
Description ✨
Solves multiple dependencies that are currently breaking Dependabot update workflows.
Issues 🐛
Closes 🙅♀️
DEV-XXX:
Related 🤝
Testing ⚗️
Automated tests ⚙️️
Manual testing 👷♂️
Screenshots 📸
Additional notes 🗒️
Summary by CodeRabbit